-
Notifications
You must be signed in to change notification settings - Fork 147
feat: use hatchling, drop setuptools #1323
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
f7a43c1 to
27cb9ad
Compare
0d9175a to
e7d0b60
Compare
4eed882 to
2063c20
Compare
3785324 to
8a314c4
Compare
8a314c4 to
0294825
Compare
7a2f2b1 to
28ab5b2
Compare
a5a9aca to
b4b12ce
Compare
shiftinv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good! I spent some time looking into hatchling + versioningit, but haven't fully reviewed everything just yet, so here are some preliminary comments:
dd74f06 to
2db92a2
Compare
6594b5b to
ff625d2
Compare
ff625d2 to
853d9ed
Compare
91ddca7 to
ddaa1f3
Compare
ddaa1f3 to
dbc301d
Compare
Without this setting, installation of disnake from a source distribution without source control metadata will fail, because there will be no way to determine the version of disnake without that metadata. Using a fallback ensures that in such an installation method (such as from a GitHub source archive), disnake can still be installed, though the version metadata may be inaccurate. Compared to not allowing installation at all, this is a better user experience, despite the incorrect version information.
1d0b983 to
a4ed52b
Compare
cc098a7 to
ea9d6a3
Compare
| steps: | ||
| - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0 | ||
| with: | ||
| fetch-depth: '0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assuming actions/checkout supports it, --shallow-since would be an option to at least limit this a little
shiftinv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in any case, looks good to me!
closes #664